Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP DONOTMERGE] Initial spike for cascading Hive Cluster Installation errors to RP #3013

Conversation

tsatam
Copy link
Collaborator

@tsatam tsatam commented Jul 6, 2023

Which issue this PR addresses:

Spike implementation for ARO-3525

What this PR does / why we need it:

Initial spike implementation for cascading known install failures from the installer through Hive to the RP.

This spike implements the ARM RequestDisallowedByPolicy error case.

For now, this spike implementation should not be merged as-is. The implementation is a proof-of-concept, and dependent on other moving parts (Hive deployment configuration) in order to work.

Test plan for issue:

Attempt to create a cluster against an RP running this code, pointed to a Hive cluster configured with the ARO-specific install log regex rules. This cluster should be deployed in an environment where in-installer resource provisioning (e.g. bootstrap/master node VMs) would be blocked by an Azure policy.

Is there any documentation that needs to be updated for this PR?

TBD

@tsatam tsatam force-pushed the ARO-3525/spike-hive-provisionfailed-condition-handling branch 2 times, most recently from da4b1d6 to 80ed4a4 Compare July 6, 2023 20:17
pkg/hive/manager_test.go Outdated Show resolved Hide resolved
pkg/hive/manager.go Outdated Show resolved Hide resolved
@schiruma schiruma force-pushed the ARO-3525/spike-hive-provisionfailed-condition-handling branch from 97692f1 to 035e8c8 Compare July 28, 2023 20:59
@tsatam tsatam closed this Aug 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants